-
Notifications
You must be signed in to change notification settings - Fork 0
Conversation
Merge dev to motif_estimation to bring branch up to date
3c4f733:
Additionally, the documentation was updated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work so far but apart from the typos can you please check if your scripts work if the provided bed has a header.
bin/get_best_motif.py
Outdated
parser.add_argument("output", help="Output file") | ||
parser.add_argument("num", help="Number of motifs parsed from file") | ||
args = parser.parse_args() | ||
return args | ||
|
||
# write lines of file till certain line (MOTIF + [num]) | ||
''' | ||
The script has to functions: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
two
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed: 8389226
bin/get_best_motif.py
Outdated
The script has to functions: | ||
1. Writing lines of file till certain line (MOTIF + [num]) | ||
2. Converting GLAM2 output to minimal meme-format | ||
@params meme STING Path to 'meme' file generated from Meme suite |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
STRING
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed: 8389226
bin/get_best_motif.py
Outdated
1. Writing lines of file till certain line (MOTIF + [num]) | ||
2. Converting GLAM2 output to minimal meme-format | ||
@params meme STING Path to 'meme' file generated from Meme suite | ||
@parmas output STING Output file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
STRING
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed: 8389226
Script can now take BED-files with and without header as input. See: 2fca158 |
bin/bed_to_fasta.R
Outdated
opt <- parse_args(opt_parser) | ||
|
||
#' Splitting BED-files depending on their cluster. | ||
#' The Sequences of each cluster are writen as an FASTA-file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
written
fixes #11 |
Fix for issue '#11'.
Improved documentation for the script 'bed_to_fasta.R'.